Skip to content

[ML] Data frame analytics: Fixes wizard for data view with runtime fields#242557

Merged
jgowdyelastic merged 4 commits intoelastic:mainfrom
jgowdyelastic:fixing-dfa-wizard-for-data-views-with-runtime-fields
Nov 12, 2025
Merged

[ML] Data frame analytics: Fixes wizard for data view with runtime fields#242557
jgowdyelastic merged 4 commits intoelastic:mainfrom
jgowdyelastic:fixing-dfa-wizard-for-data-views-with-runtime-fields

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic commented Nov 11, 2025

Fixes #242481

If a data view contains a runtime field, it will be added twice. This has become a problem after a recent Data Grid change.

@jgowdyelastic jgowdyelastic self-assigned this Nov 11, 2025
@jgowdyelastic jgowdyelastic added release_note:fix :ml v9.3.0 v9.2.1 Feature:Data Frame Analytics ML data frame analytics features backport:version Backport to applied version labels labels Nov 11, 2025
@jgowdyelastic jgowdyelastic marked this pull request as ready for review November 11, 2025 15:20
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner November 11, 2025 15:20
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@peteharverson peteharverson changed the title [ML] Fixing DFA wizard for data view with runtime fields [ML] Data frame analytics: Fixes wizard for data view with runtime fields Nov 11, 2025
Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with data views with runtime fields and LGTM

Copy link
Copy Markdown
Contributor

@alvarezmelissa87 alvarezmelissa87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ⚡

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 5.4MB 5.4MB +67.0B

History

cc @jgowdyelastic

@jgowdyelastic jgowdyelastic merged commit 3f68ad6 into elastic:main Nov 12, 2025
15 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.2

https://github.com/elastic/kibana/actions/runs/19291316842

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 12, 2025
…elds (elastic#242557)

Fixes elastic#242481

If a data view contains a runtime field, it will be added twice. This
has become a problem after a recent Data Grid
[change](elastic/eui#9030).

(cherry picked from commit 3f68ad6)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
9.2

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 12, 2025
…ime fields (#242557) (#242634)

# Backport

This will backport the following commits from `main` to `9.2`:
- [[ML] Data frame analytics: Fixes wizard for data view with runtime
fields (#242557)](#242557)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"James
Gowdy","email":"jgowdy@elastic.co"},"sourceCommit":{"committedDate":"2025-11-12T08:37:49Z","message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:Data
Frame Analytics","backport:version","v9.3.0","v9.2.2"],"title":"[ML]
Data frame analytics: Fixes wizard for data view with runtime
fields","number":242557,"url":"https://github.com/elastic/kibana/pull/242557","mergeCommit":{"message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/242557","number":242557,"mergeCommit":{"message":"[ML]
Data frame analytics: Fixes wizard for data view with runtime fields
(#242557)\n\nFixes https://github.com/elastic/kibana/issues/242481\n\nIf
a data view contains a runtime field, it will be added twice. This\nhas
become a problem after a recent Data
Grid\n[change](https://github.com/elastic/eui/pull/9030).","sha":"3f68ad6d92d30a120932eb4ae466c3f9de442f9e"}},{"branch":"9.2","label":"v9.2.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: James Gowdy <jgowdy@elastic.co>
@mistic mistic removed the v9.2.1 label Nov 12, 2025
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…elds (elastic#242557)

Fixes elastic#242481

If a data view contains a runtime field, it will be added twice. This
has become a problem after a recent Data Grid
[change](elastic/eui#9030).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:Data Frame Analytics ML data frame analytics features :ml release_note:fix v9.2.2 v9.3.0

Projects

None yet

6 participants